-
-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[18.0][MIG] hr_expense_cancel #281
base: 18.0
Are you sure you want to change the base?
Conversation
eb7cb1c
to
0b8f9c0
Compare
63f2a04
to
1650425
Compare
1650425
to
7fb11e9
Compare
This module lets to cancel and correct expenses. It adds a cancel button on the expense sheet that undo reconciliations and delete payments and journal entries.
The multi own account test has been removed as it is no longer possible in odoo core to register the payment of several expense sheet reports at the same time. Co-Authored By: Stefan Ungureanu aiendry-aktivsoftware
In this version, if a product doesn't have cost, you are only able to enter total amount, so the test should be adjusted to this condition.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: hr-expense-17.0/hr-expense-17.0-hr_expense_cancel Translate-URL: https://translation.odoo-community.org/projects/hr-expense-17-0/hr-expense-17-0-hr_expense_cancel/
Currently translated at 100.0% (4 of 4 strings) Translation: hr-expense-17.0/hr-expense-17.0-hr_expense_cancel Translate-URL: https://translation.odoo-community.org/projects/hr-expense-17-0/hr-expense-17-0-hr_expense_cancel/it/
f3e2b0d
to
734c83a
Compare
734c83a
to
7f238e3
Compare
in V18 you can only create moves for submitted sheets odoo/odoo@f892800#diff-42c75e4d29952aadf0abdd1bfe7e066bef3db1275c16d64dd3f3b96a332a2ff3
11066e9
to
369eab5
Compare
2a46bc1
to
029b582
Compare
029b582
to
aa5d2fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional review ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR has the |
Hi @Saran440 , could you please check this pr? thanks! |
No description provided.